Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make naersk’s nixpkgs input follow our nixpkgs input #152

Conversation

Jayman2000
Copy link
Contributor

This pull request gets rid of an unnecessary extra instance of Nixpkgs. Having less instances of Nixpkgs will (hopefully) make things faster.

@lucperkins
Copy link
Member

@Jayman2000 Makes sense to me. If you can fix the merge conflict introduced by your other PR, it's good to go.

@Jayman2000 Jayman2000 force-pushed the naersks-nixpkgs-follows-our-nixpkgs branch from 2704140 to 2f9c510 Compare January 9, 2025 15:55
This change gets rid of an unnecessary extra instance of Nixpkgs. Having
less instances of Nixpkgs will (hopefully) make things faster.
@Jayman2000 Jayman2000 force-pushed the naersks-nixpkgs-follows-our-nixpkgs branch from 2f9c510 to 90b2058 Compare January 9, 2025 15:56
@Jayman2000
Copy link
Contributor Author

@Jayman2000 Makes sense to me. If you can fix the merge conflict introduced by your other PR, it's good to go.

Done.

@lucperkins lucperkins merged commit c41b748 into DeterminateSystems:main Jan 9, 2025
16 checks passed
@Jayman2000 Jayman2000 deleted the naersks-nixpkgs-follows-our-nixpkgs branch January 9, 2025 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants